Skip to content

fix(slots): warn on container config drift#880

Merged
thinmintdev merged 1 commit into
mainfrom
fix/863-config-drift
Jun 17, 2026
Merged

fix(slots): warn on container config drift#880
thinmintdev merged 1 commit into
mainfrom
fix/863-config-drift

Conversation

@thinmintdev

Copy link
Copy Markdown
Contributor

Summary

  • add an opt-in slot config-drift payload by comparing live container argv with the freshly rendered restart argv
  • surface drift through slot status, API serialization, and CLI status output without adding shell-outs to the slot list poll path
  • harden drift comparison by pinning renderer flag spellings and treating --model symlink/realpath spellings as the same file

Verification

  • uv run --extra dev ruff check src/hal0/slots/manager.py tests/providers/test_container.py tests/slots/test_manager.py
  • uv run --extra dev pytest tests/api/test_slots_routes.py tests/cli/test_slot_status.py tests/providers/test_container.py tests/slot_view/test_aggregator.py tests/slots/test_manager.py -q (206 passed, 1 Starlette/httpx deprecation warning)

Notes

  • Singleton container_provider access remains intentional house style in SlotManager; a broader injection refactor should handle all call sites together.
  • CT105 deploy smoke still needs to confirm an untouched slot reports no drift in the real runtime environment.

@thinmintdev thinmintdev force-pushed the fix/863-config-drift branch from cfb7440 to 91f03ac Compare June 17, 2026 01:24
@thinmintdev thinmintdev force-pushed the fix/863-config-drift branch from 91f03ac to 3dd8dcc Compare June 17, 2026 01:25
@thinmintdev thinmintdev merged commit c8f0397 into main Jun 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant